Refactoring Percy and SDK Tool #101
Conversation
…Percy integration
…instructions and error handling
… into bstack-sdk
…d instructionBuilder file
…check in setUpPercyHandler
…and improve error handling
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Co-authored-by: Copilot <[email protected]>
… into bstack-sdk
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the SDK tool architecture by introducing a modular, feature-specific organization for Percy visual testing integration. It separates concerns between different Percy integration modes (Web, Automate, and combined BrowserStack SDK), improves code reusability, and establishes a clear structure for handling various testing configurations.
Key changes include:
- Modular reorganization of SDK utilities into feature-specific directories
- New Percy integration handlers for different modes (Web, Automate, SDK combined)
- Enhanced type safety and schema validation
- Centralized constants and common utilities
Reviewed Changes
Copilot reviewed 45 out of 45 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/tools/simulate-percy.ts | New tool for simulating Percy visual changes for new users |
| src/tools/sdk-utils/percy-web/ | Percy Web standalone integration handlers and configurations |
| src/tools/sdk-utils/percy-automate/ | Percy Automate integration handlers and configurations |
| src/tools/sdk-utils/percy-bstack/ | Percy + BrowserStack SDK combined integration handlers |
| src/tools/sdk-utils/common/ | Shared utilities, types, schemas, and constants |
| src/tools/sdk-utils/bstack/ | BrowserStack SDK-only integration handlers |
| src/tools/percy-snapshot-utils/ | Utilities for Percy snapshot management and test file detection |
| src/tools/percy-sdk.ts | Main Percy setup tool registration |
| src/tools/bstack-sdk.ts | Refactored BrowserStack SDK tool registration |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
No description provided.